Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update maplibre-gl to 5.0 #218

Merged
merged 5 commits into from
Jan 6, 2025
Merged

Update maplibre-gl to 5.0 #218

merged 5 commits into from
Jan 6, 2025

Conversation

HarelM
Copy link
Collaborator

@HarelM HarelM commented Jan 1, 2025

This add some missing attributes from the map options object and the navigation control.
The main changes are in the preserveDrawingBuffer, which was moved to an internal object.
Added projection settings as well.
Removed some noise in the console log of the unit-tests.
Added globe control directive and globe example.

@HarelM HarelM requested a review from marcjulian January 1, 2025 10:08
@HarelM HarelM requested review from hheexx and birkskyum January 2, 2025 06:59
Copy link
Collaborator

@marcjulian marcjulian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. The new globe control is awesome

@@ -10,7 +10,7 @@ import { MapComponent, LayerComponent } from '@maplibre/ngx-maplibre-gl';
"
[zoom]="[15]"
[center]="[-122.486052, 37.830348]"
[preserveDrawingBuffer]="true"
[canvasContextAttributes]="{preserveDrawingBuffer: true}"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps we could add this as breaking change to the changelog?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, except we don't really follow semver since we mainly follow angular versioning...
I also changed the changelog editing to be more automated so that only when trying to release a new version the changelog will be updated as part of the "bump version PR" workflow.

@HarelM
Copy link
Collaborator Author

HarelM commented Jan 6, 2025

I'll be merging this now. We can continue the discussion here on how to release it, which addition the changelog needs and what version number we should give.

Copy link
Member

@birkskyum birkskyum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds good - the git diff changes look good to me, but I haven't tried it in an app.

@hheexx
Copy link
Collaborator

hheexx commented Jan 6, 2025

I can try it in production once its released to npm

@HarelM HarelM merged commit 88b4d79 into main Jan 6, 2025
2 checks passed
@HarelM HarelM deleted the maplibre-5 branch January 6, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants